Skip to content

Cherry-pick from upstream: [clang] Add an API to retrieve implicit constructor arguments.#1240

Merged
rjmccall merged 1 commit intoswiftlang:apple/stable/20200108from
martinboehme:cherry-pick-4c09289
May 19, 2020
Merged

Cherry-pick from upstream: [clang] Add an API to retrieve implicit constructor arguments.#1240
rjmccall merged 1 commit intoswiftlang:apple/stable/20200108from
martinboehme:cherry-pick-4c09289

Conversation

@martinboehme
Copy link

Summary:
This is needed in Swift for C++ interop -- see here for the corresponding Swift change:

swiftlang/swift#30630

As part of this change, I've had to make some changes to the interface of CGCXXABI to return the additional parameters separately rather than adding them directly to a CallArgList.

Reviewers: rjmccall

Reviewed By: rjmccall

Subscribers: cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D79942

Summary:
This is needed in Swift for C++ interop -- see here for the corresponding Swift change:

swiftlang/swift#30630

As part of this change, I've had to make some changes to the interface of CGCXXABI to return the additional parameters separately rather than adding them directly to a `CallArgList`.

Reviewers: rjmccall

Reviewed By: rjmccall

Subscribers: cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D79942
@martinboehme
Copy link
Author

@swift-ci Please test

@martinboehme
Copy link
Author

@swift-ci Please test Windows

1 similar comment
@martinboehme
Copy link
Author

@swift-ci Please test Windows

@martinboehme
Copy link
Author

@rjmccall Can you review?

Copy link

@rjmccall rjmccall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, still looks good to me.

@martinboehme
Copy link
Author

@rjmccall Thanks! Can you or someone else with write access merge?

@rjmccall rjmccall merged commit 018c4f9 into swiftlang:apple/stable/20200108 May 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants